Skip to content

[PC-1088] Portenta C33 (Post-launch review) #912

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

Jorgetrujilloroman
Copy link
Collaborator

@Jorgetrujilloroman Jorgetrujilloroman commented Mar 14, 2023

What This PR Changes

-Small typos and grammar fixes on the product page.

Contribution Guidelines

Small typos and grammar fixes on the product page.
Copy link
Contributor

@jcarolinares jcarolinares left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check the order of table footnotes too

image

@gatsby-cloud
Copy link

gatsby-cloud bot commented Mar 14, 2023

✅ docs-content deploy preview ready

Minor corrections to the titles of figures and tables in the datasheet and some typographical errors.
Deleting unnecessary previous-added table subtitles
Copy link
Contributor

@jcarolinares jcarolinares left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please proceed with the small change/correction

Co-authored-by: Jorge Trujillo Román <124149761+Jorgetrujilloroman@users.noreply.github.com>
Copy link
Contributor

@jcarolinares jcarolinares left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Once the CI has finished do the merge and check when it published that the change are on the server with no problems please.

thank you!

@Jorgetrujilloroman Jorgetrujilloroman merged commit f0f6f38 into main Mar 14, 2023
@Jorgetrujilloroman Jorgetrujilloroman deleted the jorgetrujilloroman/portenta-c33/content-review branch March 14, 2023 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants